Skip to content

feat: add t3 project move command with --dry-run flag#2467

Open
shoaib050326 wants to merge 2 commits intopingdotgg:mainfrom
shoaib050326:issue-2358-dry-run-move
Open

feat: add t3 project move command with --dry-run flag#2467
shoaib050326 wants to merge 2 commits intopingdotgg:mainfrom
shoaib050326:issue-2358-dry-run-move

Conversation

@shoaib050326
Copy link
Copy Markdown

@shoaib050326 shoaib050326 commented May 2, 2026 •
edited by cursor Bot
Loading

Summary

  • Adds t3 project move <project> <path> command that physically moves a project workspace directory and updates the workspace root in the orchestration system
  • Adds --dry-run flag that previews the move operation without making any changes
  • Validates source exists, target parent exists and is writable, and target doesn't already exist

Changes

  • apps/server/src/cli.ts: New projectMoveCommand following existing patterns (add/remove/rename)
  • apps/server/src/cli.test.ts: 4 new tests covering success, dry-run no-op, missing source, and existing target

Testing

  • All 13 CLI tests pass
  • Lint and typecheck pass with no new warnings

Risk/Rollback

  • Low risk: additive feature, no changes to existing commands
  • Rollback: remove projectMoveCommand and revert projectCommand subcommands array

Closes #2358

Note

Medium Risk
Adds a filesystem-moving CLI command that renames directories and updates orchestration metadata; mistakes or edge cases could relocate/delete user workspaces despite some validation and rollback.

Overview
Adds a new t3 project move <project> <path> subcommand that physically renames a project’s workspace directory and then dispatches a project.meta.update to persist the new workspaceRoot (with a best-effort rollback if dispatch fails).

The command includes --dry-run output and validation for missing source, missing/non-writable target parent, existing target path, and conflicts with another active project already registered at the target location. Tests were extended to cover successful moves, dry-run no-op behavior, and key failure cases.

Reviewed by Cursor Bugbot for commit cf4ca9b. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Add project move subcommand to the t3 CLI with --dry-run support

  • Adds a t3 project move <project> <path> command in cli.ts that renames a project's workspace directory and updates its workspaceRoot metadata via a project.meta.update dispatch.
  • Validates that the source directory exists, the target parent is writable, the target path does not already exist, and no other active project is registered at the target path.
  • --dry-run flag prints a preview of the source and target paths without making any filesystem or metadata changes.
  • On dispatch error after a successful rename, the command attempts to rename the directory back to its original path.
  • Test coverage in cli.test.ts covers success, dry-run, and all validation failure cases.

Macroscope summarized cf4ca9b.

@chatgpt-codex-connector
Copy link
Copy Markdown

chatgpt-codex-connector Bot commented May 2, 2026

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 2, 2026 •
edited
Loading

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 5c881978-0571-4405-96f8-2940b5c9c908

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels May 2, 2026
macroscopeapp[bot]
macroscopeapp Bot reviewed May 2, 2026
View reviewed changes
Comment thread apps/server/src/cli.ts Outdated
Comment thread apps/server/src/cli.ts
cursor[bot]
cursor Bot reviewed May 2, 2026
View reviewed changes
Comment thread apps/server/src/cli.ts Outdated
@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp Bot commented May 2, 2026 •
edited
Loading

Approvability

Verdict: Needs human review

This PR introduces a new CLI command (project move) with filesystem operations to relocate project directories. New user-facing features and workflows warrant human review regardless of code quality or test coverage.

You can customize Macroscope's approvability policy. Learn more.

@shoaib050326 shoaib050326 force-pushed the issue-2358-dry-run-move branch from 55b0f5a to 1d722c2 Compare May 2, 2026 14:19
@github-actions github-actions Bot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels May 2, 2026
cursor[bot]
cursor Bot reviewed May 2, 2026
View reviewed changes
Comment thread apps/server/src/cli.ts
@shoaib050326 shoaib050326 force-pushed the issue-2358-dry-run-move branch from 1d722c2 to 36025a7 Compare May 2, 2026 15:04
cursor[bot]
cursor Bot reviewed May 2, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 36025a7. Configure here.

Comment thread apps/server/src/cli.ts
feat: add t3 project move command with --dry-run flag
381a39e 
- Adds t3 project move <project> <path> to physically move workspace dirs
- Adds --dry-run flag that previews without making changes
- Validates source exists, target parent exists+writable, target not taken
- Reuses existing project.meta.update orchestration command for metadata
- Adds 4 tests: success, dry-run no-op, missing source, existing target

Closes pingdotgg#2358
@shoaib050326 shoaib050326 force-pushed the issue-2358-dry-run-move branch from 36025a7 to 381a39e Compare May 2, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@macroscopeapp macroscopeapp[bot] macroscopeapp[bot] left review comments

@cursor cursor[bot] cursor[bot] left review comments

Assignees

No one assigned

Labels

size:L 100-499 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add --dry-run flag to move command

1 participant

@shoaib050326